-
Notifications
You must be signed in to change notification settings - Fork 534
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[EVM-696] eth_FeeHistory #1669
[EVM-696] eth_FeeHistory #1669
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
Left some comments. Please take a look at Lint errors it reported as well. It looks good. |
@owenwahlgren please check linting issues |
thanks for the help :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One linting issue pending |
LGTM, Just fix lint errors |
Description
Implementation of EIP1559
eth_feeHistory
into the newgasprice
packageExample call:
curl http://localhost:10002/ -X POST -H "Content-Type: application/json" --data '{"method":"eth_feeHistory","params":[4, 5, [25, 75]],"id":1,"jsonrpc":"2.0"}'
Response:
{"jsonrpc":"2.0","id":1,"result":{"OldestBlock":"0x1","BaseFeePerGas":["0x34d670c0","0x2e3ba2a8","0x2e3ba2a8"],"GasUsedRatio":["0x0","0x0"],"Reward":[["0x0","0x0"],["0x0","0x0"]]}}
Changes include
Checklist
Testing